Skip to content

chore: gitignore .claude/settings.local.json#153

Merged
obra merged 1 commit intoobra:mainfrom
shaanmajid:fix/gitignore-local-settings
Dec 9, 2025
Merged

chore: gitignore .claude/settings.local.json#153
obra merged 1 commit intoobra:mainfrom
shaanmajid:fix/gitignore-local-settings

Conversation

@shaanmajid
Copy link
Contributor

@shaanmajid shaanmajid commented Dec 8, 2025 •
edited
Loading

.claude/settings.local.json contains machine-specific paths and permissions. Cloning the repo fresh shows warnings for paths that don't exist.

This adds it to .gitignore and removes it from tracking.

Fixes #154

@shaanmajid
chore: gitignore .claude/settings.local.json
8c18403 
This file contains machine-specific paths and permissions that cause
warnings when cloning the repo fresh.
@coderabbitai
Copy link

coderabbitai bot commented Dec 8, 2025 •
edited
Loading

Walkthrough

A local Claude settings configuration file was removed from the repository, and the .gitignore was updated to prevent future commits of this file type, eliminating any locally-enforced permissions rules and directory configurations.

Changes

Cohort / File(s) Summary
Configuration cleanup
\.claude/settings.local.json		 Deleted local settings file that contained permissions configuration (allow, deny, ask rules) and directory exclusions.
Git ignore update
\.gitignore		 Added .claude/settings.local.json entry to prevent version control tracking of local settings files.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A settings file bids its farewell,
Local permissions no longer to dwell,
In .gitignore it shall rest,
Clean repos are truly the best!
✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding .claude/settings.local.json to .gitignore while removing the file from version control.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@shaanmajid shaanmajid mentioned this pull request Dec 8, 2025
Closed
@obra obra merged commit 4ac6783 into obra:main Dec 9, 2025
1 check passed
@obra
Copy link
Owner

obra commented Dec 9, 2025

thanks

@coderabbitai coderabbitai bot mentioned this pull request Dec 15, 2025
Closed
8 tasks
mthalman pushed a commit to mthalman/superpowers that referenced this pull request Dec 19, 2025
@shaanmajid @mthalman
chore: gitignore .claude/settings.local.json (obra#153)
6615c75 
This file contains machine-specific paths and permissions that cause
warnings when cloning the repo fresh.
# Conflicts:
#	.claude/settings.local.json
pcvelz pushed a commit to pcvelz/superpowers that referenced this pull request Jan 30, 2026
@shaanmajid
chore: gitignore .claude/settings.local.json (obra#153)
2e22b4c 
This file contains machine-specific paths and permissions that cause
warnings when cloning the repo fresh.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

.claude/settings.local.json should be gitignored

2 participants

@shaanmajid @obra